Merge branch 'wip/hadess/fix-remote-filechooser-main' into 'master'

Fix crasher/warnings when browsing remote filesystems

Closes #2482

See merge request GNOME/gtk!1490

Merge branch 'align-docs-fixes' into 'master'

gtk: Tweak GTK_ALIGN_BASELINE docs

See merge request GNOME/gtk!1518

gtk: Tweak GTK_ALIGN_BASELINE docs

Merge branch 'migration-can-target-docs' into 'master'

Docs: Refer to GtkWidget::can_target in migration guide

See merge request GNOME/gtk!1517

Docs: Refer to GtkWidget::can_target in migration guide

This makes the docs consistent with the renaming changes in commit
ef982b7d46125f206b59ae4222d26947bda86246.

Merge branch 'vfl-error' into 'master'

Fix message for VFL parser errors

See merge request GNOME/gtk!1514

Fix message for VFL parser errors

The loop is iterating over the wrong variable, and it's reimplementing
g_strfill() in a less efficient way.

Merge branch 'wip/ricotz/annotations' into 'master'

Really fix annotation for GtkDropTarget.get_gtypes()

See merge request GNOME/gtk!1510

Fix annotation for GdkContentFormats.get_gtypes()

Really fix annotation for GtkDropTarget.get_gtypes()

Update Japanese translation

Update Japanese translation

Merge branch 'master' into 'master'

builder-tool: rename show-close-button to show-title-buttons

See merge request GNOME/gtk!1497

Merge branch 'wip/tbaederr/fallthrough' into 'master'

Replace fallthrough comments with G_GNUC_FALLTHROUGH

See merge request GNOME/gtk!1505

Replace fallthrough comments with G_GNUC_FALLTHROUGH

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1504

stack: Make static analyzer happy

pathbar: Don't do the same thing twice.

Don't do the same thing twice.

reftest: Plug memleak

treeview: Don't assign value twice.

cellarea: Be very clear

The static analyzer needs to know we absolutely DO NOT want to use this
return value.

builder-tool: Don't allow property to be both resize and shrink

Otherwise builder-tool crashes when you do

  <property name="shrink" name="resize">1</property>

Thanks to the static analyzer for figuring that one out.

applicationaccels: Use g_renew()

liststore: Fix gtk_list_store_iter_is_valid()

The iter may be invalid, so we may not read from it.

testsuite/gtk/treemodel tests this and valgrind is shouting about it,
but it never crashed until I just ran it...

This bug is from 2004 and the test is from 2007. I guess invalid memory
accesses don't get caught by CI much.

testtreeview: Fix 19 year old use-after-free

treepath: Use g_renew()

testsuite: Avoid passing NULL to strcmp()

Use g_strcmp0() instead.

treeview: Don't initialize variable twice

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1502

build: Add -Wnull-dereference

Sprinkle various g_assert() around the code where gcc cannot figure out
on its own that a variable is not NULL and too much refactoring would be
needed to make it do that.

Also fix usage of g_assert_nonnull(x) to use g_assert(x) because the
first is not marked as G_GNUC_NORETURN because of course GTester
supports not aborting on aborts.

icontheme: Remove unused parameter

It's only ever set to NULL (and then dereferenced, too)

treerbtree: Use for loops

Makes the code more readable.
Also does the NULL check for the initial element, which was misisng
before.

notebook: Don't crash on drags not coming from a notebook

icontheme: Don't crash on icon lookup failure

device: Handle missing axis

rbtree: Simplify macros

* NODE_FROM_POINTER() can not be NULL

* shuffle NODE_TO_POINTER() code for readability so that the NULL check
  is on the outside and not the cast

rbtree: Simplify code

This makes it easier to understand for gcc (and hopefully humans, too)
that by rotating we essentially just swap `node` and `p`.

build: Add more useful warning flags

I found those on the interwebs and decided they are useful.

mountoperation-x11: Fix copy/paste thinko

tests: Add a test for changing the hovered widget

The add/remove one crashes after a few enter/leave events.

imcontextwayland: Fix copy/paste error

textview: Fix copy/paste error

Merge branch 'patch-1' into 'master'

gdk_monitor_get_model: Fix a typo in the function documentation

See merge request GNOME/gtk!1499

gdk_monitor_get_model: Fix a typo in the function documentation

Update POTFILES.in

Merge branch 'doc-fixes' into 'master'

Small documentation fixes

See merge request GNOME/gtk!1500

Fix annotation for GtkDropTarget.get_gtypes()

The argument name must match in the documentation, and the `allow-none`
annotation is deprecated, and should be replaced by `optional` in this
case.

Remove incorrect rename-to annotations

The `rename-to` annotation is used to "shadow" a symbol with another
one, which means both symbols need to exist. It can't be used to rename
a symbol to something else.

Fix introspection warnings for GtkDragIcon

Add missing annotation for GtkIconTheme

Update Japanese translation

Update Japanese translation

Merge branch 'gbsneto/gtkeventcontrollerfocus' into 'master'

build: Install gtkeventcontrollerfocus.h

See merge request GNOME/gtk!1498

build: Install gtkeventcontrollerfocus.h

It's a public class, and is included by gtk.h. Ship it.

builder-tool: rename show-close-button to show-title-buttons

For GtkHeaderBar show-close-button has been renamed to
show-title-buttons for GTK4. This commit adds this fixup to the builder
tool

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1496

droptarget: Fast-path local value load

This way, we can ensure that for local same-type drops the GValue
is set when ::enter is emitted.

This is the common case for dnd between widgets inside larger
applications, so it's worth it to speed it up.

drop: Don't use g_object_get()

dnd: Fix some documentation gotchas

Merge branch 'wip/otte/dnd' into 'master'

DND: Sync drop target

See merge request GNOME/gtk!1495

drop: Remove unused convenience APIs

With the new event controllers, these are unused.

themes: Set proper foreground color for DND icons

calendar: Use a drag source

dnd: Don't create custom color drag icons

... use the default one. It looks much better.

dragicon: Add gtk_drag_icon_create_widget_for_value()

... and use it to set a drag icon.

contentformats: Constify some functions

dragicon: Change how to acquire drag icons

Before, gtk_drag_icon_new_for_drag() allowed creating new drag icons.
This could cause multiple drag icons to exist for a single drag.

Now, gtk_drag_icon_get_for_drag() makes sure that only one drag icon is
created.

dragicon: Add GtkDragIcon::child

gdk: Make gdk_drop_status() take preferred action

This allows textview/text dnd to properly display a MOVE icon when in
the widget the drag started from but a COPY icon otherwise.

colorswatch: Switch to dragged color during dnd

This is really simple to implement now, so do it.

gtk-demo: Add a simple peg solitaire DND demo

droptarget: Redo

This is a huge reorganization of GtkDropTarget. I did not know how to
split this up, so it's unfortunately all one commit.

Highlights:

- Split GtkDropTarget into GtkDropTarget and GtkDropTargetAsync
  GtkDropTarget is the simple one that only works with GTypes and offers
  a synchronous interface.
  GtkDropTargetAsync retains the full old functionality and allows
  handling mime types.

- Drop events are handled differently
  Instead of picking a single drop target and sending all DND events to
  it, every event is sent to every drop target. The first one to handle
  the event gets to call gdk_drop_status(), further handlers do not
  interact with the GdkDrop.
  Of course, for the ultimate GDK_DROP_STARTING event, only the first
  one to accept the drop gets to handle it.
  This allows stacking DND event controllers that aren't necessarily
  interested in handling the event or that might decide later to drop
  it.

- Port all widgets to either of those
  Both have a somewhat changed API due to the new event handling.
  For the ones who should use the sync version, lots of cleanup was
  involved to operate on a sync API.

placessidebar: Don't use the GdkDrag

First, it should have been a GdkDrop, but even then, proper DND code
should not rely on internals.

It's only been used in an unused signal emission anyway.

notebook: Make dnd page switching a drop controller

This untangles tab dnd from page switching.

textview: Move drop scrolling to drop motion controller

Don't confuse the drop target with it.

treeview, iconview: Don't return the drop target

Not a good idea to hand internal event controllers out to public API.

testsuite: Use g_assert() in tests

Merge branch 'piotrdrag/placessidebar-typos' into 'master'

placessidebar: Fix a couple of typos and thinkos in menu labels

See merge request GNOME/gtk!1494

placessidebar: Fix a couple of typos and thinkos in menu labels

Merge branch 'kill-grabs-2' into 'master'

Kill grabs

See merge request GNOME/gtk!1492

Drop gtk_grab_add/remove from public api

The only form in which we still allow grabs to take place
is with modal toplevels.

Drop gtk_grab_get_current

Another grab-related api that we are not using.

Drop device grabs

We were not using this api at all, so lets drop it.

Merge branch 'matthiasc/for-master' into 'master'

inspector: Fix the "Software GL" switch

See merge request GNOME/gtk!1491

inspector: Fix the "Software GL" switch

This was crashing.

filechooser: Fallback if content-type unavailable

Fallback to fast-content-type if the content-type attribute isn't
available, as is the case for most remote filesystems.

Closes: #2482

filechooser: Fix crash when file has no content-type

This might happen for slow filesystems where a fast-content-type might
be provided instead. Don't try to manipulate that content_type if it's
NULL, otherwise we'll either throw warnings (at best) or crash (at
worse).

Conflicts:
gtk/gtkfilechooserwidget.c

Update POTFILES.in and POTFILES.skip

Merge branch 'matthiasc/for-master' into 'master'

Matthiasc/for master

See merge request GNOME/gtk!1488

Remove a stray g_object_unref

Events are no longer objects, so this does not do
any good. Use gdk_event_unref instead.

Avoid a crash in css font features values

Update Dutch translation

Merge branch 'patch-1' into 'master'

Fix a typo in gtkplacessidebar.c

See merge request GNOME/gtk!1486

Fix a typo in gtkplacessidebar.c

Some mistyped vim command got into the code in commit 60d2813247831adf9642f87ad2bf597280d2f31a.

Merge branch 'matthiasc/for-master' into 'master'

Fix a leftover occurrence of ::expand

See merge request GNOME/gtk!1485

Fix a leftover occurrence of ::expand

Update Persian translation

Merge branch 'matthiasc/for-master' into 'master'

gesture: Actually track targets

See merge request GNOME/gtk!1484

gesture: Actually track targets

Due to a mismerge, this wasn't actually working,
and was causing criticals from gtk_widget_has_grab
when dragging windows.

builder-tool: Fix a thinko

We can't rely on the pspec for a removed property.
This code worked until I actually removed the properties.

Update the tests to reflect this.